Skip to content

fix(docs): Correct README comments in logcdf documentation #5782

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

fix(docs): Correct README comments in logcdf documentation #5782

wants to merge 2 commits into from

Conversation

ChaitraTM
Copy link

Resolves #5715.

Description

What is the purpose of this pull request?

This pull request:

  • {{TODO: add description describing what this pull request does}}

Related Issues

Does this pull request have any related issues?

This pull request:

  • Moves misplaced reference section from Lines 142-145 to Line 234 in the README.md file.

  • Removes an unused #include <math.h> header from example.c.

  • Adds a missing closing </section> tag and trailing comment in the README.md file.

  • Resolves Address commit comments (commit d269f48) #5715

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

@stdlib-js/reviewers

@stdlib-bot stdlib-bot added First-time Contributor A pull request from a contributor who has never previously committed to the project repository. Needs Review A pull request which needs code review. Good First PR A pull request resolving a Good First Issue. labels Mar 4, 2025
@gururaj1512
Copy link
Member

Similar to #5783; This PR includes a submodule.

@kgryte kgryte added Needs Changes Pull request which needs changes before being merged. and removed Needs Review A pull request which needs code review. labels Mar 17, 2025
@kgryte kgryte added the autoclose: Git History Pull request which should be auto-closed due to a messed-up Git history. label Mar 31, 2025
@stdlib-bot
Copy link
Contributor

Thank you for working on this pull request. However, we cannot accept your contribution due to Git history issues.

Some common issues include:

  • Merge conflicts
  • Inappropriate commit messages
  • Unrelated commits
  • Including changes from merged branches that weren't intended for this PR

We recommend opening a new pull request with only the intended changes.

Thank you for your interest in stdlib, and we look forward to your future contributions.

@stdlib-bot stdlib-bot closed this Mar 31, 2025
@ChaitraTM ChaitraTM deleted the fix/5715-fix-readme-comments branch May 6, 2025 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

autoclose: Git History Pull request which should be auto-closed due to a messed-up Git history. First-time Contributor A pull request from a contributor who has never previously committed to the project repository. Good First PR A pull request resolving a Good First Issue. Needs Changes Pull request which needs changes before being merged.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Address commit comments (commit d269f48)

4 participants

@ChaitraTM @gururaj1512 @stdlib-bot @kgryte